Skip to content

Separate endecode from endecryption#11

Merged
juny2400 merged 3 commits into
mainfrom
separate-endecryption
May 18, 2026
Merged

Separate endecode from endecryption#11
juny2400 merged 3 commits into
mainfrom
separate-endecryption

Conversation

@juny2400
Copy link
Copy Markdown
Contributor

@juny2400 juny2400 commented May 13, 2026 •
edited
Loading

  1. Remove build warnings
  • name preset shadowed
  • unused parameter
  • std::array uninitialized
  1. Separate endecode from endecryption
  • expose encode and decode to public scope from encryptor and decryptor
  1. Add inplace decrypt and its benchmark

@juny2400 juny2400 marked this pull request as ready for review May 13, 2026 05:27
@juny2400 juny2400 requested a review from a team May 13, 2026 05:27
@juny2400 juny2400 force-pushed the separate-endecryption branch from 611d62b to 4797b9e Compare May 13, 2026 07:12
@juny2400 juny2400 requested review from gnuykeat and jonghoo-lee May 18, 2026 05:03
@juny2400 juny2400 merged commit 988bbb2 into main May 18, 2026
8 checks passed
@juny2400 juny2400 deleted the separate-endecryption branch May 18, 2026 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leejonghyeong leejonghyeong leejonghyeong approved these changes

@gnuykeat gnuykeat gnuykeat approved these changes

@jonghoo-lee jonghoo-lee Awaiting requested review from jonghoo-lee

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@juny2400 @leejonghyeong @gnuykeat